testgtk: Remove an unused variable

Update German translation

Update Indonesian translation

Remove an unused field

Drag contexts are objects, so there is no need to carry a
manual refcount around.

x11: Fix initial drag cursors

Under X, we were not setting the right drag cursor initially,
because at current_action == action == 0, initially. Fix this
by explicitly using the right cursor when grabbing.

Update Swedish translation

Update Italian translation

emoji: Use the right gettext domain

Otherwise, strings are not translated.

https://bugzilla.gnome.org/show_bug.cgi?id=786841

Update Polish translation

Update Friulian translation

Update Kazakh translation

Update Brazilian Portuguese translation

gtkplacessidebar: implement libcloudproviders support

Add integration of the libcloudproviders DBus API to the
GtkPlacesSidebar by showing name and sync status of the cloud providers.
The exported menu is rendered as a GtkPopover.

The sidebar will be updated if the list of cloudproviders changes e.g.
by adding or removing an account. If any cloud provider changes detailed
information like sync status only the individual sidebar row gets
updated.

Co-authored-by: Carlos Soriano <csoriano@gnome.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786123

ComboBox: Use iter before popdown() may invalidate

Bad actors, such as our very own FileChooserButton, may connect to the
:popped-up property and alter the model as the menu becomes in/visible.

We were getting an iter to the model while popped-up, then doing
popdown(), then using the iter, which may have just been invalidated by
the errant notify::popped-up handler. If so, we quickly crash fatally.

This is clearly bonkers, but until such patterns are removed, we have to
work around them. So, set_active() from the clicked item while it is
known to be valid, by moving the call to set_active() before popdown().

While here, change set_active_iter(iter) to set_active_internal(path) to
avoid pointlessly going through the iter to get the path we already have

https://bugzilla.gnome.org/show_bug.cgi?id=729651

Expander: Also hide child GdkWindow when collapsed

Just adding/removing to/from the BoxGadget is not sufficient; that
leaves the GdkWindow hanging around, taking input, changing the cursor,
and all sorts of other nefarious shenanigans.

Resolve by ensuring the child’s GdkWindow is unmapped if collapsed.

Note: the reflexive solution is just to set_visible(child, expanded),
but it is best to avoid messing with the child’s :visible property.

https://bugzilla.gnome.org/show_bug.cgi?id=776937

ComboBox: Clarify documentation of get_active_iter

https://bugzilla.gnome.org/show_bug.cgi?id=618160

Popover: Include window shadows in overshoot calcs

.update_position() enforces that non-Wayland platforms must position a
Popover within its parent Window. We use the allocation of the Window
to translate the position and check for overshoot on each of its sides.
Calling Widget.get_allocation() of a CSD Window includes its shadows.

But shadows were not excluded from the area in which we can position.
Thus, Popovers could get positioned in the shadow of CSD windows, where,
at least on X11, no input is received. Therefore, positioning a Popover
over a shadow meant its child widgets within that area became unusable.

Fix by calling Window.get_shadow() and including it in the overshoot on
each side. This adjusts for how the allocation includes shadows, making
overshoots with and without shadows the same. Thus, we avoid considering
shadows as viable for positioning, favouring a side where input works.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

testpopover: Use HeaderBar to get CSD decorations

This helps test whether the Popover positioning gets messed up by the
presence of CSD shadow or other accessories around the content area.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

Update Chinese (Taiwan) translation

Update Swedish translation

Update Swedish translation

Update POTFILES.in and POTFILES.skip

SidebarRow: Add Unmount tooltip to eject_button

It was reported that the lack of a tooltip made its purpose unclear.
This can be solved by just copying PlacesViewRow’s eject_button tooltip.

https://bugzilla.gnome.org/show_bug.cgi?id=766909

GtkFileChooserNativeQuartz: restore parent focus after closing dialog

Tested for both modal and non-modal dialogs

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: improve support for file filters

Instead of using conditional compilation, use respondsToSelector to
check at runtime for setAccessoryViewDisclosed.

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: add support for Mountain Lion and Mavericks

https://bugzilla.gnome.org/show_bug.cgi?id=784723 introduced support for
native file chooser dialogs on macOS, but due to the use of generics in
the patch, there will be compilation errors on pre-Xcode 7 platforms,
such as Mountain Lion and Mavericks.

I strongly recommend to revert this patch when the oldest supported
macOS release is bumped to Yosemite (10.10).

https://bugzilla.gnome.org/show_bug.cgi?id=785306

gdk: wayland: always set buffer scale on size update

We're only updating this when we have an EGL window. This means we
have the wrong scaling factor when using Vulkan.

https://bugzilla.gnome.org/show_bug.cgi?id=786492

Update Friulian translation

Update Italian translation

Update Italian translation

getting_started: Sync filenames & section headings

The order of Examples 3 and 4 was swapped, so the headings updated
themselves, but the filenames were left reflecting the old order.

https://bugzilla.gnome.org/show_bug.cgi?id=786553

3.22.19

Handle emoji data change in emoji chooser

Update the settings schema to follow the change in Emoji data,
and make the emoji chooser code handle the new format.

Add shortnames to emoji data

The json file is from

https://github.com/emojione/emojione/tree/master/extras/alpha-codes

It is under the MIT license.

Don't consume too many key releases

The Emoji input support in GtkIMContextSimple was
unintentionally consuming too many key release
events. Fix this.

https://bugzilla.gnome.org/show_bug.cgi?id=786594

Update Spanish translation

Update Nepali translation

Update German translation

Update German translation

Update Nepali translation

Update Kazakh translation

Update French translation

Update French translation

Update German translation

Update Croatian translation

Update Croatian translation

gtkbutton: Avoid keyboard grabs

Seems to be there for the sole purpose of ensuring the button
shall receive the key release on keyboard-triggered activation.

For the cases where this makes sense (eg. comboboxes, menubuttons,
...) gtk+ already does ensure the menu is popup after key release.
This makes the grab pretty useless, and there's many other cases
where it doesn't make sense (eg. button being activated
programmatically from an event handler in another widget).

Fixes button activation unintendedly triggering shortcut inhibition
on wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=786480

gdkwindow: Avoid re-setting the opaque region if it doesn't change

This avoids, at least, needless chatter with the compositor and the X
server in X11's case.

https://bugzilla.gnome.org/show_bug.cgi?id=786469

Update Slovak translation

Updated Czech translation

GDK-Win32: Fix AeroSnap on HiDPI

Commit 1d0fad3 revealed that there were some assumptions made that were
actually to compensate for the bug fixed by that commit, so we need to
remove those assumptions as they would result in AerSnap to not work
properly on HiDPI screens.

Also re-do how we set the x and y positions of our GdkWindow, so that we
are more consistent across the board when we go between a GDK window
coordinate and a Windows API window cooredinate.

This would also simplify the code a bit.

https://bugzilla.gnome.org/show_bug.cgi?id=785999

gdk/x11: Extract GdkDeviceTool tool ID from "Wacom Serial IDs" property

This property contains 5 integers, of which the last 2 respectively
contain the tool serial number and tool ID. We were only extracting the
first so far, but GdkDeviceTool also has API getters for the latter,
which remained 0.

https://bugzilla.gnome.org/show_bug.cgi?id=786400

Update Galician translation

GdkDisplay: Call the correct push|pop_error_trap()

It is wrong to assume all Displays are of the same class as the default.

https://bugzilla.gnome.org/show_bug.cgi?id=784016

ScrolledWindow: Bin unreachable ternary branches

We already checked that the child is not NULL above.

Coverity CID 1454191

Widget: Fix outdated comments about tooltips

They are not usually yellow anymore, the previous advice about how to
style them was for pre-3.20 versions, and the immediate replacement (CSS
class .tooltip) does not seem ready for primetime.

https://bugzilla.gnome.org/show_bug.cgi?id=784421

gtk/gtkemojichooser.c: Don't use g_autoptr()

We can't use g_autoptr() on non-GCC-style compilers.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Updated Lithuanian translation

Remove an erroneous unref

This was leftover by accident.

Redo emoji data slightly

No longer store variation sequences explicitly. Instead, put a 0
in the sequence where the modifiers will be inserted. This is more
compact, and it allows us to put variations directly into the
recent section. Update the type of the recent-emoji setting to
match these changes.

Don't update headings in an idle

It is not necessary since flow box invalidation is
all synchronous currently. If that ever changes,
this code will need adjustment.

testpopover: Sync initial vals of controls & props

The ComboBoxes were initially empty, rather than reflecting the initial
values of the properties. The CheckButtons were only correct by chance.
Fix this by setting the initial values on the widgets and binding them
to the properties using SYNC_CREATE, so the two are always synced up.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

configure: Add missing quotes to unbreak an AS_IF

Without quoting, the comma in AC_MSG_ERROR was wrongly interpreted as
separating the if and else paths.

https://bugzilla.gnome.org/show_bug.cgi?id=786287

Update Spanish translation

Update Galician translation

Update Turkish translation

emoji: Fix a leak

We were leaking the list of recently used Emoji here.
Pointed out by Timm Bädert.

emoji: Start search on the first character

There is no need to wait for a longer prefix here.
We already display all the content anyway, it will
only get less.

Drop a redundant helper

We already had a function that does exactly what
is needed here.

Remove an unused field

Nothing uses bubble_window, so it can go.

Update Brazilian Portuguese translation

Update Brazilian Portuguese translation

InfoBar: Fix wrong/uninit var use in code example

https://bugzilla.gnome.org/show_bug.cgi?id=786144

Updated Serbian translation

Updated Serbian translation

Adwaita: Sync CSS files with current SASS

A recent commit for emoji also updated seemingly unrelated parts of the
generated CSS files, presumably due to other things that changed in
master. The CSS files should be kept in sync with their SASS sources.

LevelBar: Avoid unnecessarily copying allocation

Just use the original allocation, instead of copying it and passing that

Update Indonesian translation

Update Indonesian translation

convert-emoji: Add missing braces

convert-emoji: Don’t pointlessly do rest of loop

skip can never become FALSE again, so break as soon as it becomes TRUE.

Update Polish translation

emojichooser: Add context to the Objects category string

To disambiguate it from Objects in the Inspector.

Update POTFILES.skip

widget-factory: Add a use of show-emoji-icon

This also adds a testcase for nested popovers that was
missing in widget-factory so far.

entry: Add support for an Emoji chooser

Add an "Insert Emoji" item to the context menu in entries.
We also add a show-emoji-icon property, which when set to
TRUE, will add an icon that can be clicked to bring up
the Emoji chooser.

Fix a problem with focus handling in modal popovers

When the popover is dismissed, we return the focus to
where it came from. However, by using gtk_widget_grab_focus,
we were messing up the selection if that widget happens to
be an entry. Special-case GtkEntry and use
gtk_entry_grab_focus_without_selecting to avoid this issue.

Add an Emoji chooser widget

Add a popover that shows color Emoji, with a search entry.
The recently-used Emoji are stored in a GSetting.

Add Emoji data

The json file is imported from the (MIT-licensed) emoji.json[0] node
module, which generates it from the emoji list published by the
Unicode Consortium.

This commit also adds a little tool to convert the data into
a compact GVariant, and the result of that conversion, which is
added to libgtk as a resource. The following commits will make use
of it.

[0] https://github.com/amio/emoji.json

Update Polish translation

Update Galician translation

Update Galician translation

Update Friulian translation

Duplicate the TIC strings before NULLing the GObject data.

NULLing the data strings in GObject frees and invalidates the pointers
returned by g_object_get_data so we need our own copy of the values.

Revert "levelbar: Don't underallocate block widgets"

This reverts commit 78b4885fe8850e132d8bb06df8ab90ac6c2033e0.

That commit causes a block always to be shown at the min side, even when
the value is 0, in which case, no fill should be shown.

https://bugzilla.gnome.org/show_bug.cgi?id=783649

gtk/gtkfilechoosernativewin32.c: Declare variables at top-of-block

Since GTK+-3.22.x is not advertised to use C99, make sure that the build
works on pre-C99 compilers such as pre-2013 Visual Studio.

Avoid the use of g_autofree

We can't use this in code that may be built on win32,
so lets just not use it at all.

3.22.18

Layout: Fix documentation on getting the GdkWindow

Members are not public like they used to be, so accessors must be used.

Revert "Widget: Remove obsolete hack-arounds for HandleBox"

This reverts commit 12805a4fbfa66b10efbe48c03595b0d5b7608d0e.

I must have been looking at the wrong tree because HandleBox is only
deprecated in GTK+ 3, not removed.